Bhavpreet: listing bidding dashboard demand chart#4172
Bhavpreet: listing bidding dashboard demand chart#4172one-community merged 21 commits intodevelopmentfrom
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
PR verified end-to-end: UI flows work as expected and CSS is consistent, responsive, and theme-aware (light/dark parity). Disabled/active states, toasts, charts, and maps render correctly with proper spacing, typography, and contrast; no layout shifts or console errors observed. Overall, no visual regressions—behavior and styling meet the spec. |
Aditya-gam
left a comment
There was a problem hiding this comment.
- Tested this PR following the instructions in the PR description.
- The charts work correctly in dark and light modes.
- The filters modify the charts according to the selections made.
- In the dark mode, in the filters section, the Current Metric's value and the
Choose metrictitle are not readable.
TestVideo.mov
the readble issue that you mention is not the part of the pr, the pr is suppose to be for the charts |
Aswin20010
left a comment
There was a problem hiding this comment.
Reviewed PR #4172 (Word Cloud Charts Addition). Verified the new Word Cloud visualization renders correctly under /lbdashboard, functions in dark mode, and aligns with existing dashboard styling. Confirmed chart responsiveness, data rendering, and no UI or performance regressions — approved the PR.
Aditya-gam
left a comment
There was a problem hiding this comment.
- Tested this PR following the instructions in the PR description.
- The charts work correctly in dark and light modes.
- The filters modify the charts according to the selections made.
TestVideo.mov
|
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Bhavpreet,
I have reviewed your PR locally, though the functionality works as expected in both light and dark mode. I did identify a UI issue : As you can see in both the charts below the highest value is getting trimmed off the higher the graph goes in the chart (Could be due to the are around the chart title). Please look into it.

smohansatyaram
left a comment
There was a problem hiding this comment.
Hi Bhavpreet, I tested this PR locally and reviewed the demand charts on the LB Dashboard. The charts render smoothly, respond to the filter selections, and work correctly in both light and dark modes. The transitions between datasets look good and the feature behaves as expected. I did notice that the top values on the charts appear slightly clipped when the graph peaks, so it may help to adjust the padding or margin above the chart area. Other than that, the implementation looks solid.
bhanuanishakkineni
left a comment
There was a problem hiding this comment.
Hello Bhavpreet,
I have checked this PR and tested all the new requirements through logged in in as admin.
The UI is working fine and responsive
Charts are working fine and are dynamic an smooth.
And current metrics are clearly displayed
All the above features are working fine in both light and dark modes.
PR.4172.testing.mp4
debadyuti23
left a comment
There was a problem hiding this comment.
Hi Bhavpreet I have reviewed your changes, all the dashboards, features render correctly as shown in the video.
PR_4172.mp4
|
Hi Bhavpreet, Tested your PR. All the charts are loading perfectly with various options. Transition is smooth. One small thing noticed here is , in Dark mode the selected dropdown like Demand or Vacancy is not highlighted as it is highlighted in Light mode, only the dropdown icon is highlighted in Dark mode. Rest everything looks good. Screen.Recording.2025-12-10.120552.mp4 |
4c953c4
the isssue that you mention is very rare, although i have added extra padding on top
|
|
Thank you all, merging! |




















Description
Added the demand charts

Or Implements # (WBS)
Related PRS (if any):
checkout to this branch and the development branch in the backend
…
Main changes explained:
How to test:
npm installand...to run this PR locallyScreenshots or videos of changes:
###Video of the feature
Screen.Recording.2025-10-04.222647.mp4
Note:
Include the information the reviewers need to know.